feat(fogclaw): E2E baseline tests, CI workflow, and OpenClaw API fix#3
Closed
sidmohan0 wants to merge 3 commits into
Closed
feat(fogclaw): E2E baseline tests, CI workflow, and OpenClaw API fix#3sidmohan0 wants to merge 3 commits into
sidmohan0 wants to merge 3 commits into
Conversation
Validated OpenClaw Dashboard automation feasibility, CLI agent interaction patterns, and plugin update mechanism. All spike questions resolved: text/role selectors work for Playwright, `openclaw agent --json` returns structured responses, plugin update is a single CLI command. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
4-milestone plan: infrastructure setup, CLI-driven scanning layer tests, access request backlog E2E, browser visual evidence with video recording. Hybrid CLI + browser architecture. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…plugin API - Add Playwright E2E test suite with isolated OpenClaw profile per run - Fix tool registration: schema → parameters, handler → execute (OpenClaw API compat) - Add GitHub Actions E2E workflow with Discord notifications - Add global-setup/teardown for automated gateway lifecycle - Tests cover all 3 scanning layers + access request backlog + dashboard Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
4 tasks
Contributor
Author
|
Closing this after porting the load-bearing part: the tool-contract fix ( The E2E suite, CI workflow, and Discord notifier here are five months stale: |
sidmohan0
added a commit
that referenced
this pull request
Jul 2, 2026
…esh (#5) * fix: harden allowlist patterns against ReDoS, enforce full-match semantics Port the datafog-python 4.7.0 allowlist hardening: reject quantified groups containing nested quantifiers at config time (catastrophic backtracking on attacker-influenced entity text), cap pattern length at 512 chars, anchor patterns so a partial match never suppresses a finding, and skip pattern matching for entities longer than 512 chars (fail-safe: the finding is kept). * fix: register tools with parameters/execute to match OpenClaw plugin contract All 6 tools were registered with schema:/handler:, which the OpenClaw runtime does not recognize — tool calls failed before reaching the model (verified against upstream docs/plugins/building-plugins.md; execute receives (toolCallId, params)). Ported from PR #3, which discovered the breakage during E2E testing in February. Also declares the six tools in the manifest contracts.tools block, now required for tool discovery, and bumps the manifest version to 0.4.0. * chore: fix npm audit vulnerabilities, bump sharp and vitest npm audit reported 10 vulnerabilities (2 critical). audit fix cleared the protobufjs/rollup/tar transitives; the rest were the vitest 2.x dev chain, cleared by vitest 4. sharp 0.34.5 -> 0.35.3. onnxruntime pins stay as-is: gliner 0.0.19 (still latest) expects onnxruntime 1.19.x internals, and the test suite mocks GLiNER so a runtime ABI break would not be caught here. Adds the openclaw.compat block and Node >=22.19 engine floor now required by the plugin runtime. Version to 0.4.0. * docs: changelog for 0.4.0
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
global-setup.ts/global-teardown.tsmanage a fresh OpenClawe2e-testprofile per run — no interference with local dev stateschema:→parameters:andhandler:→execute:across all 6 tool registrations to match the actual OpenClaw plugin contractnotify-discord.shposts pass/fail embeds to the team webhookTest plan
ANTHROPIC_API_KEYandDISCORD_WEBHOOK_URLsecrets set in repo🤖 Generated with Claude Code